const qualify some function parameters. (#229)

This is the clang-tidy check readability-non-const-parameter.

refactor "Declarator and assignment can be joined." (#227)

* refactor "Delarator and assignment can be joined."

Instances involving switch statements or goto statemens were skipped.
jeeps was skipped as well.

* fix pointer formatting case.

* more whitespace corrections.

Merge pull request #228 from tsteven4/gdb

fix irrelevant booboo in 23cb08f17.

fix irrelevant booboo in 23cb08f17.

This only mattered if NEW_STRINGS wasn't defined,
and it always is, except if running cppcheck.

Merge pull request #226 from tsteven4/resetb4read

refactor cases of "Declarator is reassigned in all paths before being read"
when the declarator and the subsequent assignment can be joined.

recover some lost comments.

refactor cases of "Declarator is reassigned in all
paths before being read"
when the declarator and the subsequent assignment can be joined.

Merge pull request #224 from tsteven4/unusedinit

limit scope of loop vars, prefer prefix operators.

use prefix increment on enums in brauniger_iq, garmin_txt, igc.

limit scope of index variables in for loops.

Merge pull request #223 from tsteven4/nullptr

clang-tidy readability-delete-null-pointer

clang-tidy readability-delete-null-pointer

formatted by hand.

Merge pull request #222 from tsteven4/cmake2

catch CMakeLists up.

catch CMakeLists up.

Merge pull request #221 from gpsbabel/cleanups

Cleanups: remove jeeps gpsinput.*

Remove gpsinput.{cc,h}

Remove unused Jeeps file handling old Jeeps Almanac files and other
esoterica uninteresting to us.

Merge pull request #137 from viettaml/master

Fix redundant file rename on Windows

Merge pull request #218 from gpsbabel/cleanups

Cleanups

Remove more C casts

Remove some senseless casts to ints, explicitly avoid const-ified queue
iterators.

More queue casting

This doesn’t _really_ help improve the code; this is to get a passing
presubmit from codify.

Merge pull request #220 from tsteven4/cmake

Add CMakeLists.txt.

Add CMakeLists.txt.

This is experimental at this stage.

downcast queue_elem to void then up to Waypoint

(yes, this is distasteful)

Fix delete/xfree mismatch in xmltag copy

Other misc minor changes tweaks identified by codacy during last commit.

Rollback erroneously committed files

git rm -r build-app-Desktop_Qt_5_9_5_clang_64bit-Debug/

(Thanx, tsteven4!)

Merge pull request #219 from tsteven4/clean888

miscellaneous cleanups & bug fixes.

miscellaneous cleanups & bug fixes.

easygps.cc: bugfix - clang-tidy check misc-sizeof-expression
garmin_fs.cc: warning cleanup - gcc warning: warning: ‘%d’ directive output may be truncated writing between 1 and 11 bytes into a region of size 3 [-Wformat-truncation=]
jtr.cc: warning cleanup - gcc warning: ‘__builtin___snprintf_chk’ output may be truncated before the last format character [-Wformat-truncation=]
jtr.cc: bugfix - the jtr writter could incorrecly print the msec portion of the creationtime by dropping the most significant digit when it was zero.
mapsource.cc: warning cleanup - binary operator acts on identical operands
random.cc: bugfix - fractional seconds were always zero.
tpo.cc: warning cleanup - gcc warning: argument 2 null where non-null expected [-Wnonnull]
xmlgeneric.cc: warning cleanup - Declarator is never used

Typo fix

waypoint -> Waypoint in last commit wasn’t saved from editor buffer

Revert stupid commits

Grrr. Github desktop works exactly like git command line doesn’t for
uncommitted files. :-/

Reduce scope of a ton of variables, misc cleanups

Codacy-inspired cleanups. Makes code look more C++-looking in many
places.

eliminate some redundant redclarations. (#217)

eliminate some uneccessary passing by value in favor of pass by const reference. (#216)

cleanup filter class issues. (#215)

Merge pull request #213 from tsteven4/constcast

use const_cast to cast away const, and don't cast away const when unnecessary

a few more const_cast eliminations.

preserve const in garmin_txt.

remove more unnecessary const_casting.

get rid of more unneeded const_cast.

get MSVC happy with util const_cast changes.

remove unnecesary const_cast in util.cc.

eliminate uneccesary const_cast in osm format.

Merge remote-tracking branch 'origin/master' into constcast

use member brace initializes and default constructors. (#214)

remove more unnecessary casting away of const.

remove unnecessary casting away of const.

clang tidy uses const_cast.

This was from the google-readability-casting check,
but only Message: C-style casts are discouraged; use const_cast
were accepted.

eliminate superfluous casting "redundant cast to the same type" (#212)

* eliminate superfluous casting "redundant cast to the same type"

this was based on clang-tidy google-readability-casting check,
but only for the Message redundant cast to the same type.
Some hand manipulation of literals was done before clang-tidy.
Some fixes were rejected for portability.

* eliminate a couple casts of integer literals to double.

* a few more casts of integer literals.

Create a filter base class and make all our filters derived classes of this base class. (#190)

clang-tidy fix modernize-deprecated-headers on windows files. (#211)

fix clang-tidy found readability-static-accessed-through-instance. (#210)

clang tidy optimizes to use parameter passing by const reference. (#209)

RUN_CLANG_TIDY_FLAGS="-header-filter=.*\\\.h -checks=-*,performance-unnecessary-value-param -fix

The suggestion "parameter '*' is passed by value and only copied once; consider moving
it to avoid unnecessary copies" was not accepted.

The suggestion "the const qualified parameter '*' is copied for each invocation;
consider making it a reference" was accepted.

The suggestion "the parameter '*' is copied for each invocation but only used as a const
reference; consider making it a const reference" was accepted.

fix missing-field-initializers clang diagnostic. (#207)

use c++11 noreturn attribute specifier instead of compiler specific attributes. (#206)

This should work for all compilers, and certainly helps out MSVC code analysis.

add version check to GPSBabel.pro, fix header dependencies. (#205)

correct header file list in GPSBabel.pro, and a warning fix. (#203)

C4146: unary minus operator applied to unsigned type, result still unsigned

MSVC warning cleanup (#202)

enhance track filter test. (#201)

cleanup defs.h orgainizing route and waypt elemements. (#200)

The following prototypes were deleted, they had no definitions:
waypt_free
waypt_dupe
waypt_new
route_add
waypt_alloc_gc_data
waypt_empty_gc_data

The following functions were never used, and they had no prototypes:
void routes_to_tracks();
void tracks_to_routes();

The following function had a repeated prototype:
route_flush

The following function is missing a prototype but is explictly listed by the using file:
update_common_traits

In addition to resolving the above issues, some sections of defs.h
were moved based on if they came from waypt.cc or route.cc,
when they were required, and grouping of similar elements.

improve travis docker script. (#199)

update Dockerfile to reduce image size. (#198)

cleanup whining about validate filter, and add test.  (#197)

CI debug, codacy excludes, obsolete file deletion. (#196)

generate codacy coverage report on travis (#195)

more excludes for codacity. (#194)

Delete nmea_cc

Scratch work file that was accidentally committed.

Check in codacy config file

Ignore deprecated and third party code.

cleanups for codacy warnings. (#192)

take advantage of C++11 delete to block a default operator, and move a bunch of loop index declarations to the loop.

reduce docker layer size in an attempt to workaround network problems. (#191)

Limit scope of variables and functions that are not meant to be shared. (#188)

restrict unshared variables and functions to file scope.

restore fiddling with -e in kml test. (#189)

this allows subsequent test errors to be counted.

Merge pull request #187 from tsteven4/style_nullptr

catch mkstyle.sh up with nullptr.

catch mkstyle.sh up with nullptr.

Merge pull request #186 from tsteven4/tidyup

Modernize with clang-tidy

clang-tidy add override specifier.

-header-filter=.*\\.h -checks=-*,modernize-use-override -fix

clang-tidy replaces C standard library headers with their C++ alternatives.

-header-filter=.*\\.h -checks=-*,modernize-deprecated-headers -fix

clang-tidy replaces integer literals that are cast to bool.

-header-filter=.*\\.h -checks=-*,modernize-use-bool-literals -fix

use clang-tidy to convert the usage of null pointer constants to use the new C++11 nullptr keyword.

-header-filter=.*\\.h -checks=-*,modernize-use-nullptr -fix

add make targets to use clang-tidy.

make use of QStringLiteral with QXmlStreamWriter. (#153)

correct spelling in document. (#184)

fix link to build log in continuous windows release.

update appveyor token instructions. (#183)

bug fixes and improvements for uploadtool. (#182)

* fix body of appveyor uploads.

* give detailed instructions on appveyor upload when GITHUB_TOKEN missing.

* pick up upstream changes to upload.sh.

Use date instead of travis/appveyor build number in pre-release names. (#180)

token massage for appveyor.

fix permissions on extract-qt-installer.

Add mac build to travis CI (#179)

Add mac build to travis CI.
Deploy mac dmg to github releases from travis.
Deploy windows installer to github releases from appveyor.

Revert "add creation of appimage."

This reverts commit 46f8c773de64761cbde22adb078a26b55e41db2e.

Revert "fix packages for travis."

This reverts commit 807e853e62d2252a3a66e6b5a9fba647f88e5c3c.

Revert "fix travis"

This reverts commit 50353d5c57d3a0d4ad1dcaf301354a2675893b61.

Revert "fix travis packages, rpath."

This reverts commit 16d14a9d462de3983232d8c8d7cd8744387a9488.

Revert "fix travis builds"

This reverts commit 3e5d604ad38573a755d5c2fdf1d9593dc6a76d1d.

Revert "have travis avoid ppa environment script."

This reverts commit f74c139d26efadfa929bcaca1aea5985fa819711.

Revert "fix up travis LDFLAGS."

This reverts commit 8229a81fa41bcccb5db5e1ffba10c2975ee750ed.

Merge pull request #176 from tsteven4/appv

Drop appveyor Qt 5.7 build, they dropped Qt 5.7 with MSVC 2015.

drop appveyor Qt 5.7 build.

fix up travis LDFLAGS.

have travis avoid ppa environment script.

fix travis builds

fix travis packages, rpath.